Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improv landing #637

Merged
merged 7 commits into from
Aug 2, 2023
Merged

Improv landing #637

merged 7 commits into from
Aug 2, 2023

Conversation

chaibax
Copy link
Contributor

@chaibax chaibax commented Aug 1, 2023

Première itération

  • Changement du lien du logo tchap qui renvoyaut vers element.io
  • Harmonisation avec le titre et slogan de tchap.beta.gouv.fr
  • Ajout d'un bouton en savoir plus qui pointe sur tchap.beta.gouv.fr

Capture d’écran 2023-08-01 à 15 09 22

Tu en penses quoi @areox-net ?

@areox-net
Copy link
Contributor

@chaibax C'est très bien. Je changerais juste 3 choses:

  • le soustitre " la messagerie instantanée..." pourrait etre pas en gras, tout en gardant cette taille de typo. Ca va faire une hierarchie de titres plus claire.
  • Le bouton se connecter devrait etre bleu Tchao Check experience as an external, lot of things should be forbidden #91
  • ajouter plus d'espace entre les deux boutons connexion et "en savoir plus". le triple de l'espace actuel.

@chaibax
Copy link
Contributor Author

chaibax commented Aug 1, 2023

Voila @areox-net
Capture d’écran 2023-08-01 à 15 42 47

@areox-net
Copy link
Contributor

Merci @chaibax, c'est mieux.
En reflechissant mieux je me dis que le "en savoir plus" devrait etre avant les bouton connexion, car c'est la suite de la phrase de description.

Aussi le sous-titre peut etre en pt plus petit finalement.

This was referenced Aug 1, 2023
@chaibax
Copy link
Contributor Author

chaibax commented Aug 2, 2023

J'ai eu des soucis avec le background svg qui comportait le drapeau FR en position fix. Il ne devrait pas etre en bas par ailleurs.

J'ai mis un autre backgroud svg, en prenant le code couleur de Tchap.beta.gouv.fr

Je trouve ca beacoup plus clean

Capture d’écran 2023-08-02 à 13 53 05

Pour rappel, il y a ca en ce moment :

Capture d’écran 2023-08-02 à 14 01 55

@chaibax chaibax requested a review from odelcroi August 2, 2023 12:03
@odelcroi
Copy link
Member

odelcroi commented Aug 2, 2023

Nice background change :) I like the clouds but I'd rather wait a bit before merge it.

Je l'ai revert et j'ai ajouté les traductions anglaises

Version to be merged :

image

@areox-net areox-net self-requested a review August 2, 2023 13:59
@odelcroi odelcroi merged commit 04bea6a into develop_tchap Aug 2, 2023
5 checks passed
@chaibax
Copy link
Contributor Author

chaibax commented Aug 16, 2023

Merci @odelcroi pour le merge. J'ai 2 commentaires :

  • Ca serait vraiment cool que le logo de la république française apparaisse en premier et pas en dernier. Ca donne plus confiance, c'est plus visible, et c'est plus proche de DSFR

  • Le nouveau background c'était pour corriger le faible niveau de contraste du footer. Le texter est à peine visible. C'est pas suffisement contrasté, et donc clairement pas accessible . Source : https://accessibilite.numerique.gouv.fr/methode/criteres-et-tests/#3 3.2 "Dans chaque page web, le contraste entre la couleur du texte et la couleur de son arrière-plan est-il suffisamment élevé (hors cas particuliers) ?"

@areox-net
Copy link
Contributor

@chaibax Tu as raison sur les deux points:

  • on utilise les deux logos ensemble aussi sur les autres pages et les mails
  • pour le footer j'avais prévu autre chose, mais on l'a jaimais fait en CSS. Faire le degradé en blanc et le texte gris foncé.

Mais je ne changerais pas le fond.
Je vais travailler dessus pour faire une nouvelle PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants